From: Florian Müllner Date: Sat, 11 Jul 2020 16:10:18 +0000 (+0200) Subject: workspacesView: Do not skip entire overview transition X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~684^2^2~139 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=9e8883c9227bc5d1fc6dbd2eb32b1204df47c9d3;p=gnome-shell.git workspacesView: Do not skip entire overview transition Since commit af543daf1c3, we skip the overview transition when the actual geometry hasn't been set yet. However with the new layout manager, the only bit that still needs the separate geometry is the transition of the view, the workspaces can do their transition just fine. https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/2969 --- diff --git a/js/ui/workspacesView.js b/js/ui/workspacesView.js index a757badbb..c74c6e9c7 100644 --- a/js/ui/workspacesView.js +++ b/js/ui/workspacesView.js @@ -551,20 +551,18 @@ class WorkspacesDisplay extends St.Widget { this.show(); this._updateWorkspacesViews(); - if (this._actualGeometry) { - for (let i = 0; i < this._workspacesViews.length; i++) { - let animationType; - if (fadeOnPrimary && i == this._primaryIndex) - animationType = AnimationType.FADE; - else - animationType = AnimationType.ZOOM; - this._workspacesViews[i].animateToOverview(animationType); - } - - if (!fadeOnPrimary) - this._syncWorkspacesActualGeometry(); + for (let i = 0; i < this._workspacesViews.length; i++) { + let animationType; + if (fadeOnPrimary && i == this._primaryIndex) + animationType = AnimationType.FADE; + else + animationType = AnimationType.ZOOM; + this._workspacesViews[i].animateToOverview(animationType); } + if (this._actualGeometry && !fadeOnPrimary) + this._syncWorkspacesActualGeometry(); + this._restackedNotifyId = Main.overview.connect('windows-restacked', this._onRestacked.bind(this));